Skip to content

docs: fix agent-team uninstall label and update testing section#87

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
docs/fix-label-and-testing-docs-5b2793f2d7d91e73
Draft

docs: fix agent-team uninstall label and update testing section#87
github-actions[bot] wants to merge 1 commit into
mainfrom
docs/fix-label-and-testing-docs-5b2793f2d7d91e73

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

What changed

Two documentation fixes surfaced during the post-merge sweep of #69.

1. README.md — wrong label in uninstall steps

The agent-team uninstall block listed state:in-progress as the seventh label to delete. That label does not exist in the repo or in the agent-team install skill. The correct seventh label is agent-team:reviewed (the label the reviewer adds when it approves a PR), which is documented correctly in catalog/agent-team/README.md prerequisites but was wrong in the top-level README.

Before:

... `plan-needed`, `impl-needed`, `review-needed`, `done`, `blocked`, and `in-progress`.
```
**After**:
```
... `plan-needed`, `impl-needed`, `review-needed`, `done`, and `blocked`, plus `gh label delete agent-team:reviewed`.

2. CONTRIBUTING.md — outdated testing section

The testing section opened with "There is no automated test harness for skills", which is incorrect. The repo ships a three-tier test suite (tests/) that runs in CI on every PR. The section now accurately describes:

  • How to run the suite (./tests/run-tests.sh)
  • The three-tier structure (invariants / skill behavior / E2E)
  • CI behavior and the fork-PR limitation (Tier 1 fails on forks because CLAUDE_CODE_OAUTH_TOKEN is not leaked)
  • Retained the manual validation steps for lock files and grep counts

Why these weren't caught earlier

The label name discrepancy predates #69. The testing section was written before the test suite was added and was never updated. Neither affects runtime behavior — they are documentation-only errors.

Checklist

  • state:in-progressagent-team:reviewed in README.md uninstall block
  • CONTRIBUTING.md testing section rewritten to match the actual test suite
  • No lock files touched, no skill instructions changed

Generated by Update Docs ·

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/update-docs.md@96b9d4c39aa22359c0b38265927eadb31dcf4e2a

- README.md: correct the 7th agent-team uninstall label from the
  non-existent `state:in-progress` to `agent-team:reviewed`, which
  matches the label set listed in catalog/agent-team/README.md

- CONTRIBUTING.md: replace "there is no automated test harness" with
  an accurate description of the three-tier test suite in tests/,
  including how to run it, what each tier covers, and the fork-PR
  CI limitation

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added automation documentation Improvements or additions to documentation labels Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants